Skip to content

Ignore simulated position updates#54

Closed
herver wants to merge 1 commit intovACDM:mainfrom
herver:ignore_simulated_pilots
Closed

Ignore simulated position updates#54
herver wants to merge 1 commit intovACDM:mainfrom
herver:ignore_simulated_pilots

Conversation

@herver
Copy link
Copy Markdown
Contributor

@herver herver commented Jan 29, 2025

Plugins get simulated traffic update even when the box is unticked in Euroscope settings (it's just a display flag). This ensures no simulated traffic update is propagated to the server

@LeoKle
Copy link
Copy Markdown
Member

LeoKle commented Jan 30, 2025

Do avoid updates from simulated traffic, I did implement the following fix: 836e044.
I think this is the easier and better solution as we prevent the update from entering the DataManager (the update list to process) and we also don't need to extend the Pilot.h struct / model.

The fix has only been deployed to develop as we were awaiting feedback on it. I'll merge it with the next upcoming changes.
Does this work for you?

@herver
Copy link
Copy Markdown
Contributor Author

herver commented Feb 1, 2025

Thanks for the pointer, I missed the fix you implemented. I gave it a try and it seems to be working just fine !

@LeoKle
Copy link
Copy Markdown
Member

LeoKle commented Feb 1, 2025

No problem, thanks for testing!

@LeoKle LeoKle closed this Feb 1, 2025
@herver herver deleted the ignore_simulated_pilots branch May 15, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants